-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
💚 ci のワークフローを集約 #162
💚 ci のワークフローを集約 #162
Conversation
Warning Rate Limit Exceeded@tatsutakein has exceeded the limit for the number of files or commits that can be reviewed per hour. Please wait 0 minutes and 52 seconds before requesting another review. How to resolve this issue?After the wait time has elapsed, a review can be triggered using the @coderabbitai review command. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit. How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. ウォークスルーこの変更では、Android、iOS、およびバックエンド向けのジョブを含むCIワークフローが導入されました。メインブランチに対するプルリクエストの同時実行制御が設定され、コードのチェックアウト、Javaのセットアップ、テストの実行、および各プラットフォームのコンポーネントのビルドが含まれています。 変更
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- .github/workflows/ci.yml (1 hunks)
Additional comments: 4
.github/workflows/ci.yml (4)
1-11: 新しいCIワークフローの名前とトリガーイベントが設定されており、
main
ブランチへのプルリクエストで動作するようになっています。また、同じPRに対する現在または以前のジョブをキャンセルするためのconcurrency
設定が追加されています。これにより、CIの実行が効率化されます。13-33: Androidジョブには、コードのチェックアウト、Javaのセットアップ、Gradleのセットアップ、ユニットテストの実行、ビルドのステップが含まれています。
actions/checkout@v4
を使用しており、lfs
がtrue
に設定されている点が適切です。Javaのバージョンも17と指定されており、Gradleのセットアップも問題ないようです。ユニットテストとビルドのコマンドも適切に設定されています。35-52: iOSジョブには、Androidジョブと同様のステップが含まれており、iOSフレームワークのビルド、iOSアプリのビルド、iOSユニットテストの実行が追加されています。
macos-latest
を実行環境として使用し、xcodebuild
コマンドを使用してiOSアプリのビルドとテストを行っています。これらのステップも適切に設定されているようです。54-73: バックエンドジョブには、Androidジョブと同様のステップが含まれており、バックエンド固有のビルドとテストのステップが追加されています。ここでも
actions/checkout@v4
とactions/setup-java@v4
が使用されており、Gradleのセットアップとユニットテストのコマンドが適切に設定されています。ただし、GradleのセットアップアクションのバージョンがAndroidジョブのv2.10.0
と異なりv2
になっています。これは意図的なものか確認が必要です。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- .github/workflows/ci.yml (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- .github/workflows/ci.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (1)
- .github/workflows/ci.yml (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- .github/workflows/ci.yml
Issue
概要
ci のワークフローを集約します。
レビュー観点
特になし
レビューレベル
レビュー優先度
参考リンク
スクリーンショット
Summary by CodeRabbit